From: Carl Lerche <me@carllerche.com>
Date: Tue, 6 May 2014 05:01:34 +0000 (-0700)
Subject: Comment cleanup
X-Git-Tag: archive/raspbian/0.35.0-2+rpi1~3^2^2^2^2^2^2^2~1086
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/success//%22http:/www.example.com/cgi/success/?a=commitdiff_plain;h=c7c6622b71f44b0468b66bec6ffff87818530485;p=cargo.git

Comment cleanup
---

diff --git a/src/cargo/core/manifest.rs b/src/cargo/core/manifest.rs
index 95ca363fe..6070d24d8 100644
--- a/src/cargo/core/manifest.rs
+++ b/src/cargo/core/manifest.rs
@@ -58,33 +58,6 @@ impl Manifest {
     pub fn get_target_dir<'a>(&'a self) -> &'a Path {
         &self.target_dir
     }
-
-    /*
-    pub fn from_serialized(path: &str, serialized: &SerializedManifest) -> CargoResult<Manifest> {
-        let (lib,bin) = normalize(&serialized.lib, &serialized.bin);
-        let &SerializedManifest { ref project, ref dependencies, .. } = serialized;
-
-        let deps = dependencies.clone().map(|deps| {
-            deps.iter().map(|(k,v)| {
-                // This can produce an invalid version, but it's temporary because this needs
-                // to be replaced with Dependency, not NameVer
-                Dependency::with_namever(&NameVer::new(k.clone(), v.clone()))
-            }).collect()
-        }).unwrap_or_else(|| vec!());
-
-        let root = try!(Path::new(path.to_owned()).dirname_str().map(|s| s.to_owned()).to_result(|_|
-            CargoError::internal(PathError(format!("Couldn't convert {} to a directory name", path)))));
-
-        Ok(Manifest {
-            root: root.to_owned(),
-            project: project.clone(),
-            lib: lib,
-            bin: bin,
-            target: "target".to_owned(),
-            dependencies: deps
-        })
-    }
-    */
 }
 
 impl Target {
diff --git a/src/cargo/core/registry.rs b/src/cargo/core/registry.rs
index ec6c9557a..b8e40a84c 100644
--- a/src/cargo/core/registry.rs
+++ b/src/cargo/core/registry.rs
@@ -8,33 +8,3 @@ use core::{
 pub trait Registry {
   fn query<'a>(&'a self, name: &str) -> Vec<&'a Package>;
 }
-
-/*
- *
- * ===== Temporary for convenience =====
- *
- */
-
-/*
-pub struct MemRegistry {
-  packages: Vec<Package>
-}
-
-impl MemRegistry {
-  pub fn new(packages: &Vec<Package>) -> MemRegistry {
-    MemRegistry { packages: packages.clone() }
-  }
-
-  pub fn empty() -> MemRegistry {
-    MemRegistry { packages: Vec::new() }
-  }
-}
-
-impl Registry for MemRegistry {
-  fn query<'a>(&'a self, name: &str) -> Vec<&'a Package> {
-    self.packages.iter()
-      .filter(|pkg| name == pkg.get_name())
-      .collect()
-  }
-}
-*/
diff --git a/src/cargo/core/resolver.rs b/src/cargo/core/resolver.rs
index 7d22a718b..0a944c980 100644
--- a/src/cargo/core/resolver.rs
+++ b/src/cargo/core/resolver.rs
@@ -3,6 +3,10 @@ use core;
 use core::package::PackageSet;
 use core::errors::CargoResult;
 
+/* TODO:
+ * - The correct input here is not a registry. Resolves should be performable
+ * on package summaries vs. the packages themselves.
+ */
 #[allow(dead_code)]
 pub fn resolve(deps: &[core::Dependency], registry: &core::Registry) -> CargoResult<PackageSet> {
     let mut remaining = Vec::from_slice(deps);